-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix ObjC Support iteration #7498
base: master
Are you sure you want to change the base?
Conversation
This needs a changelog entry |
This needs tests which actually test that the bug that the user reported is fixed. |
There is a whole suite of tests for using the obj-c API from Swift and this should be tested there, not in an unrelated place. |
@tgoyne ptal. |
Nevermind, seeing some build failures on CI. |
@tgoyne PTAL. |
@@ -4,6 +4,9 @@ x.y.z Release notes (yyyy-MM-dd) | |||
* Conform `@ThreadSafe` and `ThreadSafeReference` to `Sendable`. | |||
|
|||
### Fixed | |||
* Fix `RLMCollectionIterator` where `RLMCollectionIterator` contents were not RLMObject. | |||
This is for users using the optional `RLMSupport.swift` file. | |||
* Change default request timeout for `RLMApp` from 6 seconds to 60 seconds. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bad changeset merge.
What is the status on this? It happens for me as well. |
I'm having the same thing over and over again |
Fixes #7490.